Remove the deprecated -u flag
authorAlex Crichton <alex@alexcrichton.com>
Tue, 23 Sep 2014 14:33:23 +0000 (07:33 -0700)
committerAlex Crichton <alex@alexcrichton.com>
Thu, 25 Sep 2014 15:20:12 +0000 (08:20 -0700)
This flag has been deprecated in favor of `cargo update` for quite some time
now.

src/bin/bench.rs
src/bin/build.rs
src/bin/doc.rs
src/bin/run.rs
src/bin/test.rs
src/cargo/ops/cargo_compile.rs

index d57f114951356f4aa304bc478f87882d6ea88f0e..5448adfafac7d7e597abc50067efd90a98b5478c 100644 (file)
@@ -35,7 +35,6 @@ pub fn execute(options: Options, shell: &mut MultiShell) -> CliResult<Option<()>
     let mut ops = ops::TestOptions {
         no_run: options.flag_no_run,
         compile_opts: ops::CompileOptions {
-            update: false,
             env: "bench",
             shell: shell,
             jobs: options.flag_jobs,
index eaa433a2587c966e5c582668e075a614b3a5c379..1a977920a373fd0d833662797a9e370ac6b273cd 100644 (file)
@@ -20,7 +20,6 @@ Options:
     --features FEATURES     Space-separated list of features to also build
     --no-default-features   Do not build the `default` feature
     --target TRIPLE         Build for the target triple
-    -u, --update-remotes    Deprecated option, use `cargo update` instead
     --manifest-path PATH    Path to the manifest to compile
     -v, --verbose           Use verbose output
 ",  flag_jobs: Option<uint>, flag_target: Option<String>,
@@ -39,7 +38,6 @@ pub fn execute(options: Options, shell: &mut MultiShell) -> CliResult<Option<()>
     };
 
     let mut opts = CompileOptions {
-        update: options.flag_update_remotes,
         env: env,
         shell: shell,
         jobs: options.flag_jobs,
index c24ae47ac2de2cd8c0b8d6df4c642220d1963876..a1c0beaf23670833cb26f3d08a36ef0ecdf687da 100644 (file)
@@ -17,7 +17,6 @@ Options:
     -j N, --jobs N          The number of jobs to run in parallel
     --features FEATURES     Space-separated list of features to also build
     --no-default-features   Do not build the `default` feature
-    -u, --update-remotes    Deprecated option, use `cargo update` instead
     --manifest-path PATH    Path to the manifest to document
     -v, --verbose           Use verbose output
 
@@ -35,7 +34,6 @@ pub fn execute(options: Options, shell: &mut MultiShell) -> CliResult<Option<()>
     let mut doc_opts = ops::DocOptions {
         all: !options.flag_no_deps,
         compile_opts: ops::CompileOptions {
-            update: options.flag_update_remotes,
             env: if options.flag_no_deps {"doc"} else {"doc-all"},
             shell: shell,
             jobs: options.flag_jobs,
index 82b8eb49be66bf5f01a598b529af586e782f8d1a..afd3a27b0096d90af6e7be70b571cd39d9dbea23 100644 (file)
@@ -19,7 +19,6 @@ Options:
     --features FEATURES     Space-separated list of features to also build
     --no-default-features   Do not build the `default` feature
     --target TRIPLE         Build for the target triple
-    -u, --update-remotes    Deprecated option, use `cargo update` instead
     --manifest-path PATH    Path to the manifest to execute
     -v, --verbose           Use verbose output
 
@@ -32,7 +31,6 @@ pub fn execute(options: Options, shell: &mut MultiShell) -> CliResult<Option<()>
     let root = try!(find_root_manifest_for_cwd(options.flag_manifest_path));
 
     let mut compile_opts = ops::CompileOptions {
-        update: options.flag_update_remotes,
         env: if options.flag_release { "release" } else { "compile" },
         shell: shell,
         jobs: options.flag_jobs,
index 0355e1c1ac29629c1c8a753500571f7c08a2ad48..9f6d747382a6dc5cc9b51b1e1dc210a4b2317aff 100644 (file)
@@ -19,7 +19,6 @@ Options:
     --features FEATURES     Space-separated list of features to also build
     --no-default-features   Do not build the `default` feature
     --target TRIPLE         Build for the target triple
-    -u, --update-remotes    Deprecated option, use `cargo update` instead
     --manifest-path PATH    Path to the manifest to build tests for
     -v, --verbose           Use verbose output
 
@@ -35,7 +34,6 @@ pub fn execute(options: Options, shell: &mut MultiShell) -> CliResult<Option<()>
     let mut ops = ops::TestOptions {
         no_run: options.flag_no_run,
         compile_opts: ops::CompileOptions {
-            update: options.flag_update_remotes,
             env: "test",
             shell: shell,
             jobs: options.flag_jobs,
index b569e2c50dc2b7f0bc64d92da9df529d373e6d4b..6962f5fd97e6bda222f3952c317817fe5343b50d 100644 (file)
@@ -34,7 +34,6 @@ use util::config::{Config, ConfigValue};
 use util::{CargoResult, Wrap, config, internal, human, ChainError, profile};
 
 pub struct CompileOptions<'a> {
-    pub update: bool,
     pub env: &'a str,
     pub shell: &'a mut MultiShell<'a>,
     pub jobs: Option<uint>,
@@ -47,7 +46,7 @@ pub struct CompileOptions<'a> {
 pub fn compile(manifest_path: &Path,
                options: &mut CompileOptions)
                -> CargoResult<ops::Compilation> {
-    let CompileOptions { update, env, ref mut shell, jobs, target,
+    let CompileOptions { env, ref mut shell, jobs, target,
                          dev_deps, features, no_default_features } = *options;
     let target = target.map(|s| s.to_string());
     let features = features.iter().flat_map(|s| {
@@ -56,11 +55,6 @@ pub fn compile(manifest_path: &Path,
 
     log!(4, "compile; manifest-path={}", manifest_path.display());
 
-    if update {
-        return Err(human("The -u flag has been deprecated, please use the \
-                          `cargo update` command instead"));
-    }
-
     let mut source = try!(PathSource::for_path(&manifest_path.dir_path()));
     try!(source.update());